widget: Add a GtkBuildable::add_child implementation

Just call gtk_widget_set_parent in there.

progressbar: Remove unused defines

flowbox: don’t select when rubberbanding over nothing

When rubberbanding over an empty area, everything is selected on gesture
end, which is a bit counter-intuitive.

https://bugzilla.gnome.org/show_bug.cgi?id=780735

osx: fix build

https://bugzilla.gnome.org/show_bug.cgi?id=734946

gdkwindow: subsurface in gdk_window_get_parent()

When the GtkWidget hierarchy does not match the GdkWindow hierarchy, the
GtkWidget code may find a common ancestor that cannot be found while
traversing the GdkWindow tree using gdk_window_get_parent().

This happens with for example on Wayland, a GtkPopover has another
GtkPopover as parent, in this case, the GdkWindow parent is the root
window, whereas the GtkWidget parent is the other GtkPopover.

That confuses the gtk_widget_translate_coordinates() logic which will
bail out in this case and won't return the translated coordinates.

Make gdk_window_get_effective_parent() aware of subsurfaces and use the
transient_for which represents the actual parent (whereas the parent
might be pointing to the root window).

Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=774148

Work around a wayland crash

For some reason, we are seeing damage being NULL here.
While that should never be the case, crashing on it is
unkind and makes the Wayland experience unusable.

Document GTK_INSPECTOR_RENDERER

Also add GTK_INSPECTOR_DISPLAY to the docs at the same time.

gtk_widget_intersect: fix annotations

https://bugzilla.gnome.org/show_bug.cgi?id=773228

gskrendernodeimpl: Fix documentation argument type

GSK: Fix a bunch of warnings from the GIR scanner

arising from conflicting argument names, missing annotations, etc.

TextView: Be const-correct when passing a pointer

The TextIter is passed by pointer for efficiency. We neither need to
modify it, nor should we leave it possible to accidentally do so. So,
it should be passed as a pointer-to-const.

TextView: Get line direction in more efficient way

We do not need to go through the heavyweight process of constructing a
TextLineDisplay just to get the direction out of it, when we can simply
use TextIter API to get the text and then get its direction using Pango.

Adapted from a patch by Mehdi Sadeghi for GtkSourceView:

https://bugzilla.gnome.org/show_bug.cgi?id=779081#c20

TextBuffer: Add missing transfer full annotations

Update Polish translation

printjob: Clarify array ownership in gtk_print_job_set_page_ranges()

Add a documentation annotation saying that set_page_ranges transfers
ownership of the GtkPageRange array.
Add a g_free() call to fix a memory leak when set_page_ranges is
used repeatedly.

https://bugzilla.gnome.org/show_bug.cgi?id=780234

Avoid more compiler warnings

Avoid unused variable warnings

Avoid redefining GNU_SOURCE

Gives a compiler warning with the wrong flags.

Avoid some compiler warnings

printing: Don't hang in gtk_enumerate_printers()

Stop enumerating printers when all backends were removed.

https://bugzilla.gnome.org/show_bug.cgi?id=686838

Update Russian translation

(cherry picked from commit a7f3fd12d5c4e866a1d8e05360a7052a756ee8bd)

mir: always emit a resize when creating windows

Update German translation

glcontext: Fix buffer age comparison

The old code had a dead >=3 branch, causing flickering.

gdkdragsource: Ensure button >= 1

So the left shift later doesn't overflow.

widget: Prepend style classes to list when parsing

Since the later gtk_style_context_add_class doesn't care about the order
of the style classes, we can as well just prepend style classes to the
list and avoid the squared behavior when appending to a linked list.

builderparser: Remove pointless TagType struct

Update Indonesian translation

Scrollbar: grammar fix for previous commit

Scrollbar: Tweak intro docs a bit

Explain where the adjustment comes from, clarify some of the wording
about how its fields influence the scrollbar, and also note that the
steppers may not be present, since they aren’t in our default themes.

ScrolledWindow: Don’t check if child is Scrollable

If the child added is not a Scrollable, it gets wrapped in a ViewPort –
which is. So it is impossible to end up with a non-Scrollable child.

Just check we have /any/ child where needed, which is semantically nicer

https://bugzilla.gnome.org/show_bug.cgi?id=778853

ScrolledWindow: Streamline/clarify the intro docs

 • intro: Clarify that external policy and/or adjustments can be used.
 • add(): Don’t waffle on about having to add a ViewPort since we handle
          that transparently for the user, so they can add() any widget.
 • Adjustment stuff: most of this was repeating the docs for Scrollbar,
                     so just refer the user to that. Also, mention how
                     policies NEVER and EXTERNAL interact with all this.

https://bugzilla.gnome.org/show_bug.cgi?id=778853

Tooltip: Avoid redundant variable initialisations

has_tooltip_widget was assigned twice in immediate succession.

return_value is not used anywhere else in this function since commit
14a864c8b55dfe92f8957499f12a3f9303188a12 and does not need a default
value anymore, so move it to the inner scope and don't init to NULL.

GdkWaylandWindow: Force window titles to valid utf8

Use the new g_utf8_make_valid in case the given title is not valid utf8.
Also bump the glib requirement to 2.51.5.

label: Properly calculate half the height

Since we compare it to a double anyway, might as well divide by 2.0.

tooltip: Remove unused assignment

hide_tooltip  gets overriden in any case 2 lines down, and return_value
isn't used later in that function. The second assignment was introduced
in ef1da5f6c2450fc5f7c7de4a17114cc7507a41ad, directly below the first
assignment.

css-editor: Remove unused if statement

The static local is set to NULL and never set to anything else, so we
never use it anyway.

window: Remove pointless ternary operator

we're already checking for orientation == GTK_ORIENTATION_HORIZONTAL
right before this line, so just pass GTK_ORIENTATION_VERTICAL.

box: Make sure center_req is initialized

In case we have an invisible center widget, we never initialize it, but
later still use it.

progressbar: Remove useless if statement

Both the if and the else block contained exactly the same code.

gsk: Take scale factor into account when setting up the gl scissor

gl renderer: Fix hidpi fallback rendering

When allocating the surface, take scale into consideration.

Update Friulian translation

notebook: Remove dead assignment

GtkRecentManager: Remove superfluous local variable

GdkRenderer: Fix a compiler warning

GdkX11DeviceManager: Fix debugging output

The 2 values added in 3.22 were missing from the source_names array.

GdkMonitor: Use 1 as scale fallback value

The same value we use in gtk_widget_get_scale_factor.

css-overview: Fix inconsistent British English use

vulkan: fix compilation

entrycompletion: Remove unnecessary NULL check

completion really shouldn't be NULL at this point, especially since it
gets dereferenced higher up in the function.

calendar: Fix else branch indentations

GdkDrawingContext: Add missing break in switch statement

css-overview: Fix+Explain color expr number ranges

Last try, promise. They don’t all use 0 to 1. We should probably explain
the effects too. Hopefully this manages that while not being too verbose

Avoid a critical warning in the filechooser portal

This was showing up when cancelling a portal file chooser
in recipes.

css-overview: Elaborate how color expressions work

docs/css-overview: Fix color functions’ arg orders

shade/alpha/mix() take colour(s) and a number that is the ratio by which
to transform them. It was written here that these shall be passed in the
order (number, colour). That was wrong: they must be passed in the order
(colour[s], number) to work, and for the Inspector not to flag an error.

Update Latvian translation

docs: Remove mentions of gtk_init_with_args().

Because it has been removed: See commit 7e8eb3ddcd29ce9eecb14a56dbfeb0caf64dc772

headerbar: Fix size requisition variable mixup

Happened when porting to gtk_widget_measure.

GtkShaderBuilder Fix copy/paste error in precondition

gtk_shader_builder_add_define should check both define_name and
define_value for not-NULL and not-empty, but the second precondition
check checks define_name again for not-empty-ness.

Allow custom renderer for the inspector

If you set GTK_INSPECTOR_RENDERER to the same type of
values that GSK_RENDERER takes this can change the renderer
used for the inspector. This is useful if you're debugging
one renderer and don't want to affect the inspector.

quartz: fix build for <10.12

The style mask type was changed from NSUInteger to NSWindowStyleMask.
Use NSUInteger so it also works on older macOS.

https://bugzilla.gnome.org/show_bug.cgi?id=780019

Update Russian translation

3.89.5

Add Friulian translation

mir: start handling resize events again

filechooserwidget: Remvoe useless gtk_widget_show call

And a comment above. Even if it is not correct, it doesn't matter since
we call gtk_widget_show on it again later in that function.

modelmenuitem: Remove unnecessary gtk_widget_show calls

listbox: Avoid a gtk_widget_destroy call

filechooserwidget: Avoid a gtk_widget_destroy

filechooserwidget: Remove unnecessary gtk_widget_show calls

entry: make sure priv->current_pos is valid

We can e.g. get the entry dispose()d and a focus_out event after that
(because the toplevel unsets the focus which previously was the entry).
We then later use priv->current_pos in a call to pango API which makes
sure the given index is valid for the given layout. Since we lazily
create a GtkEntryBuffer in get_buffer() and a PangoLayout lazily in
gtk_entry_create_layout, these 2 are always valid but don't match
priv->current_pos in this situation.

Fix this by resetting priv->current-pos in dispose().

entry: Remove cursor type parameter from get_cursor_locations

We are only ever passing CURSOR_STANDARD anyway.

mir: remove unused generate_configure_event ()

mir: set placement for menu-type windows

mir: synthesize resize events

mir: ignore resize events

mir: try mir before x11

Update Slovak translation

Update Slovak translation

Updated Serbian translation

Updated Serbian translation

Update Indonesian translation

Update Indonesian translation

Updated Czech translation

Update Friulian translation

quartz: Fix another typo

quartz: Fix the build

A ; was left out inadvertently.

Add Friulian translation

mir: log additional event types

Update German translation

Improve GContentType usage

Convert to content type where needed.
Should fix various issues on Windows and OS X.

https://bugzilla.gnome.org/show_bug.cgi?id=734946

Update Swedish translation

Update Scottish Gaelic translation

(cherry picked from commit 112437f5ad4a904e874ce09922403ab44d34d94a)

Update Scottish Gaelic translation

(cherry picked from commit 2fdcdfa403ed33f7732fb5a34f80311053db37dd)

tests: Fix use of C99 inline declaration

We don’t claim to use them yet, even if we perhaps should.

testheightforwidth: cleanups

testframe: label-yalign does nothing so don’t test

Since the border node was removed, this property has no effect.

https://bugzilla.gnome.org/show_bug.cgi?id=778886

testframe: Simplify initial setting of padding

Just start with both dimensions’ paddings at 0.